Skip to content

doc: Remove unimplemented option #2330

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

ShinySaana
Copy link
Contributor

@ShinySaana ShinySaana commented Jun 4, 2025

PR Type

Documentation


Description

  • Removed SE_SAFARIDRIVER from environment variable documentation.

  • Updated English, Japanese, Portuguese, and Chinese docs for accuracy.


Changes walkthrough 📝

Relevant files
Documentation
selenium_manager.en.md
Remove SE_SAFARIDRIVER from English documentation               

website_and_docs/content/documentation/selenium_manager.en.md

  • Removed reference to SE_SAFARIDRIVER from supported environment
    variables.
  • +0/-1     
    selenium_manager.ja.md
    Remove SE_SAFARIDRIVER from Japanese documentation             

    website_and_docs/content/documentation/selenium_manager.ja.md

    • Deleted SE_SAFARIDRIVER from the list of environment variables.
    +0/-1     
    selenium_manager.pt-br.md
    Remove SE_SAFARIDRIVER from Portuguese documentation         

    website_and_docs/content/documentation/selenium_manager.pt-br.md

  • Eliminated SE_SAFARIDRIVER from environment variable documentation.
  • +0/-1     
    selenium_manager.zh-cn.md
    Remove SE_SAFARIDRIVER from Chinese documentation               

    website_and_docs/content/documentation/selenium_manager.zh-cn.md

  • Removed SE_SAFARIDRIVER from the list of supported environment
    variables.
  • +0/-1     

    Need help?
  • Type /help how to ... in the comments thread for any questions about Qodo Merge usage.
  • Check out the documentation for more information.
  • Copy link

    netlify bot commented Jun 4, 2025

    👷 Deploy request for selenium-dev pending review.

    Visit the deploys page to approve it

    Name Link
    🔨 Latest commit 277cb88

    @CLAassistant
    Copy link

    CLAassistant commented Jun 4, 2025

    CLA assistant check
    All committers have signed the CLA.

    Copy link
    Contributor

    qodo-merge-pro bot commented Jun 4, 2025

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    ⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ No major issues detected

    Copy link
    Contributor

    qodo-merge-pro bot commented Jun 4, 2025

    PR Code Suggestions ✨

    No code suggestions found for the PR.

    Copy link
    Member

    @diemol diemol left a comment

    Choose a reason for hiding this comment

    The reason will be displayed to describe this comment to others. Learn more.

    @ShinySaana, please sign the CLA.

    @ShinySaana
    Copy link
    Contributor Author

    Done!

    Copy link
    Member

    @diemol diemol left a comment

    Choose a reason for hiding this comment

    The reason will be displayed to describe this comment to others. Learn more.

    Thank you.

    @diemol diemol merged commit 6413947 into SeleniumHQ:trunk Jun 4, 2025
    3 checks passed
    selenium-ci added a commit that referenced this pull request Jun 4, 2025
    @ShinySaana ShinySaana deleted the doc/se-safaridriver-unimplemented branch June 4, 2025 13:22
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Projects
    None yet
    Development

    Successfully merging this pull request may close these issues.

    3 participants